Skip to content

fix: easy_install URL now includes /deprecated/#950

Merged
webknjaz merged 2 commits intopypa:mainfrom
rffontenelle:fix-url-easy_install
Aug 27, 2021
Merged

fix: easy_install URL now includes /deprecated/#950
webknjaz merged 2 commits intopypa:mainfrom
rffontenelle:fix-url-easy_install

Conversation

@rffontenelle
Copy link
Copy Markdown
Contributor

@rffontenelle rffontenelle commented Jul 16, 2021

Tiny fix for the homepage of the deprecated easy_install. Without /deprecated/ it ends in the 404 Not Found page of rtd.

See:

Copy link
Copy Markdown
Member

@webknjaz webknjaz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please use an intersphinx reference instead of hardcoding the URLs.

https://webknjaz.github.io/intersphinx-untangled/setuptools.rtfd.io/ should help you find the right reference.

@rffontenelle
Copy link
Copy Markdown
Contributor Author

@webknjaz Is it possible to use reference to specific section in a page? I see that :std:doc:`deprecated/easy_install` works, but won't go to the section "Natural Script Launcher".

Copy link
Copy Markdown
Member

@dukecat0 dukecat0 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There is no reference for natural-script-launcher.

@webknjaz webknjaz enabled auto-merge August 27, 2021 11:39
@webknjaz
Copy link
Copy Markdown
Member

There is no reference for natural-script-launcher.

Fair enough. This would require a PR to setuptools to add such a label.

@webknjaz webknjaz merged commit 102d895 into pypa:main Aug 27, 2021
@rffontenelle rffontenelle deleted the fix-url-easy_install branch August 27, 2021 11:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants